-
Notifications
You must be signed in to change notification settings - Fork 935
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hide tty date #486
Hide tty date #486
Conversation
2 similar comments
428a07a
to
62589c0
Compare
Bump Would really love this feature as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - want to make sure the other maintainers agree on adding yet another environment variable. I don't see a reason not to in this case.
Thanks! yes, as mentioned before, this env var is very needed for avoiding the double timestamps in Heroku, PM2, Systemd... |
Squashed commit of the following: commit 62589c0 Author: Adrian Mejia <admejiar@cisco.com> Date: Sat Aug 12 15:24:45 2017 -0400 solves merge conflict commit 55e5c5e Author: Adrian Mejia <admejiar@cisco.com> Date: Sat Aug 12 15:21:16 2017 -0400 docs commit e09dec3 Author: Adrian Mejia <admejiar@cisco.com> Date: Sat Aug 12 15:19:10 2017 -0400 cleanup commit 9dd6a2b Author: Adrian Mejia <admejiar@cisco.com> Date: Sat Aug 12 15:05:53 2017 -0400 enables DEBUG_HIDE_TTY_DATE
Thank you! Merged in 56a3853. |
This option is very needed when running Node on systems that already provides time stamps such as Heroku, PM2, systemd...
Solves #453